Skip to content

fix: requeue the request when the override has been changed#1097

Merged
zhiying-lin merged 2 commits intoAzure:mainfrom
zhiying-lin:fix-override-status
Mar 26, 2025
Merged

fix: requeue the request when the override has been changed#1097
zhiying-lin merged 2 commits intoAzure:mainfrom
zhiying-lin:fix-override-status

Conversation

@zhiying-lin
Copy link
Copy Markdown
Contributor

Description of your changes

Requeue the request when the override has been changed but the resource has not been changed.
Otherwise, the CRP will stuck in ApplyInProcess

Fixes #

I have:

  • Run make reviewable to ensure this PR is ready for review.

How has this code been tested

Added e2e tests

Special notes for your reviewer

@zhiying-lin zhiying-lin force-pushed the fix-override-status branch from 8cada6d to 399c9a2 Compare March 25, 2025 08:51
ryanzhang-oss
ryanzhang-oss previously approved these changes Mar 25, 2025
Comment thread test/e2e/placement_cro_test.go
Comment thread pkg/controllers/workgenerator/controller.go
Comment thread pkg/controllers/workgenerator/controller.go Outdated
@zhiying-lin zhiying-lin merged commit f7ea25d into Azure:main Mar 26, 2025
16 checks passed
@zhiying-lin zhiying-lin deleted the fix-override-status branch March 26, 2025 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants